Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Fault Injection from Behavior Tree Specification #23

Closed
wants to merge 8 commits into from

Conversation

rdinizcal
Copy link
Collaborator

Added two forms of fault injection from Behavior Tree Specification:

  1. Probability of inverting the signal returned from the condition testing.
    How to configure? One can configure the probability by setting a variable error in a node Condition.
    ex.: condition lc( sim_time (repeat=False, tmin=3, tmax=10), error=0.1)
    Which means that there is a 10% chance of returning an inverted result for the sim_time condition checking.

  2. Introduced delay on checking the condition.
    How to configure? One can configure the probability by setting a variable delay (in sec) in a node Condition.
    ex.: condition c_busy_lane( lane_occupied(), delay=3 )
    This simulates that the driver is taking at least 3 seconds to check if the next lane is busy.
    Obs. the total time to check for a condition is then the delay + the checking algorithm processing time.

@rdinizcal
Copy link
Collaborator Author

@rodrigoqueiroz I didn't see before that there were conflicts between this PR and the master. But since you havent been using the delay and error for fault injection, will you even merge them? (this might introduce conflicts with the fix naming PR, please let me know if I can help later on)

@mantkiew
Copy link
Collaborator

mantkiew commented Feb 6, 2025

Abandoning these changes as out of date. An example of changing the BTree grammar.

@mantkiew mantkiew closed this Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants